Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix:Added transition for smooth closing of sidebar #683

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

priyanshagra
Copy link

A meaningful title for PR, not like: made changes to xyz.md

Fixes #664

Ref #

Changes done:

  • Added transition of 0.3 sec in .main.sidebar-toggle class

Screenshots

WhatsApp.Video.2024-02-01.at.02.05.02.mp4

Preview Link(s):

✅️ By submitting this PR, I have verified the following

  • [ done] Checked to see if a similar PR has already been opened 🤔️
  • [ done] Reviewed the contributing guidelines 🔍️
  • [ done] Sample preview link added (add the link(s) for all the pages changed/updated from the checks tab after checks complete)
  • [ done] Tried Squashing the commits into one

@priyanshagra
Copy link
Author

@tachyons @tanmoysrt please check

@priyanshagra
Copy link
Author

@VaibhavUpreti please check

@tanmoysrt
Copy link
Member

@tachyons @tanmoysrt please check

Also add trasition while opening the sidebar as well.

@priyanshagra
Copy link
Author

that is not possible i thought suggest me if you can help

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

In learn documentation there is no transition delay on opening of sidebar 🐞 Bug
2 participants